refactor(tooling): recipe renames, shell completions, and tox migration stub#57
refactor(tooling): recipe renames, shell completions, and tox migration stub#57danceratopz merged 5 commits intojustfrom
Conversation
danceratopz
force-pushed
the
experiments/just
branch
from
b9a8c64 to
2f49f00
Compare
Revisit recipe names from first principles for shorter, consistent, and more intuitive CLI usage. All hyphens, no underscores. Renames: - ethereum-spec-lint → spec-lint - tests_pytest_py3 → test-tests - tests_pytest_pypy3 → test-tests-pypy - tests_benchmark_pytest_py3 → test-tests-bench - pypy3 → fill-pypy - json_loader → json-loader - markdownlint → md-lint - benchmark-gas-values → bench-gas - benchmark-fixed-opcode-cli → bench-opcode - benchmark-fixed-opcode-config → bench-opcode-config Updates all CI workflows and documentation references.
Document how to enable tab-completion for just recipes using the built-in `just --completions` command for bash, zsh, and fish.
danceratopz
force-pushed
the
experiments/just
branch
from
2f49f00 to
717e9e2
Compare
Provide a tox.ini that prints migration messages directing users to the equivalent just recipes. Helps contributors still using muscle-memory `tox -e py3` commands discover the new workflow.
Align workflow job ID with the Justfile recipe rename.
danceratopz
force-pushed
the
experiments/just
branch
from
717e9e2 to
b925e36
Compare
Group with other lint-* recipes for tab-completion consistency.
There was a problem hiding this comment.
I'd prefer we just hard break and remove tox.ini
This stub doesn't preserve backwards compatibility, and I think most people would check the readme when tox fails.
There was a problem hiding this comment.
Understand the sentiment, the screenshot in the PR description wasn't very illustrative.
The aim was to be a temporary helper (for say 2 months) for users who want to quickly know how to map their old "tox env" to the new "recipe".
That, and a friendly reminder for people switching often between branches that haven't rebased to forks/amsterdam yet.
Does that change how you feel about this temp feature? "No" is fine 😆 but I think it helps the migration path.
There was a problem hiding this comment.
I don't feel strongly about it, I just tend to try to cut as much code as possible. It is a nice helper, no argument here
There was a problem hiding this comment.
Ok, I'll leave it in. I created a reminder for the beginning of May to delete it ))
2 participants
🗒️ Description
Follow-up to the just migration PR. Three improvements:
A-test-tests→test-tests).just --completionssetup for bash, zsh, and fish in CONTRIBUTING.md.tox -e py3get a bold migration message with the equivalentjustrecipe and install link, then a non-zero exit to avoid masking real errors.Recipe renames
ethereum-spec-lintlint-speclint-*groups all linters for tab-completionmarkdownlintlint-mdlint-*groups all linters for tab-completionactionlintlint-actionslint-*groups all linters for tab-completionspec-docsdocs-specdocs-*groups all doc recipes for tab-completionfast-docsdocs-fastdocs-*groups all doc recipes for tab-completiontests_pytest_py3test-testsA-test-testsrepo label and commit scopetests_pytest_pypy3test-tests-pypytests_benchmark_pytest_py3test-tests-benchtest-testsfamilypypy3fill-pypyfill; clarifies it's a fill variantjson_loaderjson-loaderbenchmark-gas-valuesbench-gasbench-*groups benchmarksbenchmark-fixed-opcode-clibench-opcodebench-*groups benchmarksbenchmark-fixed-opcode-configbench-opcode-configbench-*groups benchmarksShell Completion Example
Tox Migration Stub Helper Example
Full Recipe List
🔗 Related Issues or PRs
Follow-up to the just migration PR:
toxwithjustethereum/execution-specs#2555✅ Checklist
```console
just static
```
Cute Animal Picture